gsk: Avoid compiler warnings without assertions

Disabling assertions was provoking some unused variable
warnings from the compiler. Avoid these.

testsuite: Reduce the use of g_assert

Stop using g_assert() in the gtk tests.

testsuite: Reduce the use of g_assert

Stop using g_assert() in the gsk tests.

Merge branch 'f16c.msvc' into 'master'

Fix F16C build/runtime detection on Visual Studio

See merge request GNOME/gtk!3429

Merge branch 'testsuite-asserts' into 'master'

testsuite: Reduce use of g_assert

See merge request GNOME/gtk!3431

testsuite: Reduce use of g_assert

Stop using g_assert() in the css tests.

testsuite: Reduce use of g_assert

Stop using g_assert() in the reftests.

testsuite: Reduce use of g_assert

Stop using g_assert() in the gdk tests.

meson.build: Check for F16C on 32-bit Visual Studio

It is also possible to enable F16C instructions on 32-bit Visual Studio builds,
so also check for the compiler's ability to build F16C code on Visual Studio
32-bit instead of just bailing out.

gsk/ngl/fp16.c: Implement runtime F16C detection on MSVC

We need to use __cpuid() to check for the presence of F16C instructions on
Visual Studio builds, and call the half_to_float4() or float_to_half4()
implementation accordingly, as the __builtin_cpu...() functions are strictly
for GCC or CLang only.

Also, since __m128i_u is not a standard intrisics type across the board, just
use __m128i on Visual Studio as it is safe to do so there for use for
_mm_loadl_epi64().

Like running on Darwin, we cannot use the alias __attribute__ as __attribute__
is also for GCC and CLang only.

meson.build: Improve F16C detection on Visual Studio

The __builtin_cpu...() intrisics are strictly for GCC/CLang-based compilers, so
don't use them in the checking code on Visual Studio.  The test code will still
compile without this change, but will certainly fail if we want it to link.

Merge branch 'ngl-shadow-fixes' into 'master'

ngl: Fix outset blurred shadows

See merge request GNOME/gtk!3427

Merge branch 'grouped-toggle-fix' into 'master'

togglebutton: Fix expected group behavior

See merge request GNOME/gtk!3425

Merge branch 'shadow-reduction' into 'master'

gsk: Don't overshadow

See merge request GNOME/gtk!3419

ngl: Fix unevenly scaled shadows

Ensure that we don't cut them off at the edges.

ngl: Don't slice unsliceable shadows

When the corners are too big to make slicing work,
don't do it, since it leads to broken results.

ngl: Fix a case of flipped shadow

In the non-sliced case, we were rendering the shadow
upside down.

ngl: Add code to dump shadows

This can be helpful in understanding shadow bugs.

Merge branch 'matthiasc/for-master' into 'master'

menubutton: Propagate focus-on-click

See merge request GNOME/gtk!3424

togglebutton: Fix expected group behavior

The expectation is that a grouped button stays
active when you click it. Grouped check buttons
behave this way, and grouped toggle buttons should
too.

node-editor: Make header buttons not steal focus

This is rarely useful, and is confusing when it
happens as side-effect of dragging the window.

icon browser: Make header buttons not steal focus

This is rarely useful, and is confusing when it
happens as side-effect of dragging the window.

widget-factory: Make header buttons not steal focus

This is rarely useful, and is confusing when it
happens as side-effect of dragging the window.

gtk-demo: Make header buttons not steal focus

This is rarely useful, and is confusing when it
happens as side-effect of dragging the window.

menubutton: Propagate focus-on-click

Propagate the focus-on-click setting to the button
inside, so that setting menubuttons as !focus-on-click
works as expected. This helps for menubuttons in
header bars, where dragging on the button will otherwise
steal focus from the content.

Update Ukrainian translation

Merge branch 'matthiasc/for-master' into 'master'

some small optimizations

See merge request GNOME/gtk!3423

ngl: Allow passing bounds to update_clip

update_clip only uses the bounds of the node that
we are currently passing. This opens the door to
updating the clip while handling a single node.

ngl: Small optimization for shadows

When 9-slicing shadows, omit the center tile when it is
entirely contained in the outline (that is not always
the case, depending on corners and offsets).

roundedrect: Speed up contains_rect and friends

gsk_rounded_rect_contains_rect was calling
gsk_rounded_rect_contains_point, which potentially
checks all four corners, for a total of up to 16
corner/point checks. But there is no need to do
more than 4 such checks to answer the question.

ngl: Small optimization

We really want equality here.

nodeeditor: Make the help window wide enough

Make the help window wide enough to show the
tables without wrapping.

Merge branch 'media-memory-leak' into 'master'

media: Fix a memory leak

See merge request GNOME/gtk!3420

media: Fix a memory leak

This was introduced in e7dc82fa32f9c115607.

gsk: Don't overshadow

Reduce the cairo shadows to the same size as their GL brethren.

Merge branch 'popover-list-lockup' into 'master'

window: Defer focus setting until after paint

Closes #3725

See merge request GNOME/gtk!3417

window: Defer focus setting until after paint

Commit 3dbf5038fab8eb0d8c11 tried to defer focus changes
until after rendering is done. But it failed to do so, since
the toplevel ::render handler is still before rendering of
popups that are attached to that toplevel. To do this
properly, we need to do it in the AFTER_PAINT frame clock
phase.

Fixes: #3725

Merge branch 'ngl-coloring-underlines' into 'master'

Optimize underlines in text

See merge request GNOME/gtk!3414

Merge branch 'no-scroll-cursor' into 'master'

scrolledwindow: Stop using scroll cursors

See merge request GNOME/gtk!3416

Merge branch 'ebassi/ci-docs' into 'master'

ci: Drop the split Fedora images

See merge request GNOME/gtk!3415

scrolledwindow: Stop using scroll cursors

We used to override cursor to use all-scroll while the
content is being scrolled. Unfortunately, there is several
problems with this:
- It is really only expected certain devices, and we don't
  have the device information on Wayland
- With the way cursor setting works in GTK4, non-NULL cursors
  of the content (eg the text views ibeam) win, making the
  scroll cursor not show up
- Under X11, we seem to miss scroll end events and then
  the scroll cursor gets stuck
Therefore, just remove this feature.

ci: Drop the split Fedora images

Now that we don't need the whole of Pandoc to build our documentation,
we can go back to a single Fedora image for our CI.

ngl: Add a program change counter

Count how often we change programs during a frame.

ngl: Optimize underlines in text

Opportunistically use the coloring program for
drawing underlines instead of the color program.
This avoids program changes in the middle of
text.

For the Emoji text scrolling benchmark, this reduces
the program changes per frame from > 1000 to around 100.

ngl: Reserve a pixel in texture atlases

This will be used for coloring from the texture.

ngl: Fix an oversight

All the rest of debug spew goes to stderr here.

gtk-demo: Make one text scroll benchmark harder

Add underlines into the mix with the Emoji heavy
text, to see if we can optimize lines.

Merge branch 'half-float' into 'master'

ngl: Use fp16 for colors

See merge request GNOME/gtk!3405

Merge branch 'font-fallback-fix' into 'master'

wayland: Improve font setting fallback more

Closes #3838

See merge request GNOME/gtk!3412

Work around compiler shortcomings on macOS

alias attributes don't work on Darwin, so
do without.

gsk: Add runtime checks for F16C

Use an IFUNC resolver to determine whether we can use
intrinsics for FP16 conversion. This requires the functions
to be no longer inline.

Sadly, it turns out that __builtin_cpu_supports ("f16c")
doesn't compile on the systems where we want it to prevent
us from getting a SIGILL at runtime.

Add an option to disable F16C

And add a compile time check for the presence of this
CPU feature.

Move color conversion up

Doing the color conversion early lets us reuse
the results whenever we do more than one draw
per node.

ngl: Use a constant for (fp16)-1

No need to convert that again for every glyph.

ngl: Reduce vertex storage

We only have one shader that uses the color2 attribute,
and it doesn't use the uv attribute, so save vertex
memory by putting those in the same space.

This reduce the per vertex space from 32 to 24 bytes.

ngl: Use fp16 for colors

This reduces the size of our Vertex struct from
48 to 32 bytes. It would be nicer if we could store
the colors in fp16 format in the rendernodes, and
avoid conversion here. But this is still good.

wayland: Improve font setting fallback more

We may get a response from the portal that contains
no useful settings at all. In that case, we should
fallback as well.

Fixes: #3838

Merge branch 'wip/carlosg/stuck-frames' into 'master'

surface: Always set PHASE_PAINT as pending when updates are scheduled

Closes #3750

See merge request GNOME/gtk!3411

surface: Always set PHASE_PAINT as pending when updates are scheduled

At times (most often when closing subsurfaces that are scheduling
relayouts) the PHASE_PAINT handling gets broken with the following
sequence:

1. Surface receives wl_callback.done for the previous frame.
   Surface is thawed.
2. A new update on the surface is scheduled. PHASE_PAINT is
   requested directly on the frame clock. priv->pending_phase is
   left unset in the surface.
3. Surface gets frozen
4. Frame clock processes the update scheduled at 2. The surface
   is frozen, so paint is prevented. PHASE_PAINT is considered
   handled.
5. Compositor emits wl_callback.done again. Surface is thawed.
6. At this point the machinery is off
   - The surface didn't paint but has pending update regions
   - priv->draw_needed is set in the toplevel and other portions
     of the widget tree
   - So queueing redraws is ineffective at eventually calling
     gdk_surface_schedule_update() again on the toplevel surface.
   - We don't paint anymore, so this broken state is not flushed
     until other subsurface changes manage to schedule the missing
     update.

To fix this, always set PHASE_PAINT in priv->pending_phase when
doing gdk_surface_schedule_update(). If the frame clock turns
around before the surface is thawed, it will still be waiting to
be processed the next iteration.

Fixes: https://gitlab.gnome.org/GNOME/gtk/-/issues/3750

Merge branch 'font-settings-fallback-4' into 'master'

wayland: Improve font setting fallback

See merge request GNOME/gtk!3407

wayland: Improve font setting fallback

When we don't get stettings from the portal, the current
fallback is 'awful fonts'. There is no need for that. Instead,
set the fallback values to grayscale antialiasing with slight
hinting.

Merge branch 'textview-last-line' into 'master'

textview: Fix a problem with the last line

Closes #3835

See merge request GNOME/gtk!3403

textview: Fix a problem with the last line

Sometimes, we missed the last line for display.

Fixes: #3835

Merge branch 'matthiasc/for-master' into 'master'

window: Actually enable the inspector by default

Closes #3831

See merge request GNOME/gtk!3401

modelbutton: Avoid a crash

We may not have a popover menu ancestor. The check for
this was forgotten in one of the branches here.

Fixes: #3831

window: Actually enable the inspector by default

427d216081ea0fd9e2045e changed the default in the schema,
but didn't handle cases where the schema isn't found.

Merge branch 'matthiasc/for-master' into 'master'

textlayout: Remove some unnecessary code

See merge request GNOME/gtk!3400

ngl: Fix an oversight

We were special-casing 2D affine transforms,
but overlooked even simpler transforms.

textlayout: Remove an unused function

textlayout: Avoid allocations for line iteration

No need to collect the lines firs, and allocate memory
for that. We can just iterate over them right away.

textlayout: Introduce some locals

No need to call _gtk_text_buffer_get_btree over and over.

textlayout: Create the cursor snapshot on demand

We don't need it all the time.

textattributes: Small docs fix

textbtree: Return tags in a GPtrArray

One of the callers prefers that, it lets us avoid
copying the array in one place, and generally makes
for better code.

textlayout: Only get selection color once

No need to do this for every paragraph that is
inside the selection.

texttag: Just use qsort

No point in trying to optimize this.

textbtree: Inline node data handling

textbtree: Speed up _gtk_text_btree_find_line_top

Another place where we can avoid allocating a stack
piecemeal.

textbtree: Don't opencode realloc

We can just use g_realloc here.

textbtree: Speed up _gtk_text_line_char_index

No need to allocate a stack piecemeal here.

textbtree: Avoid malloc in one place

textlayout: Don't call render_para too much

render_para produces no nodes for an empty paragraph.
Don't call it over and over for those.

textlayout: Remove some unnecessary code

The same field is set again 2 lines down.

Update Chinese (Taiwan) translation

(cherry picked from commit 3d3912d7f4ea27ddb9aa79f02bce6bc2d4889bb0)

Merge branch 'matthiasc/for-master' into 'master'

gsk: Rename resource paths

See merge request GNOME/gtk!3399

Merge branch 'dotted-lines3' into 'master'

Modernize error underlines

See merge request GNOME/gtk!3398

Modernize error underlines

Implement PANGO_UNDERLINE_ERROR as a dotted line, instead of
squiggly, which looks quite dated.

gsk: Move shader resources

Move the resources of each renderer to its subdirectory.
We've previously done that for the ngl renderer, but it
is better to be consistent and do it for all the renderers.

gsk: Rename resource paths

Make the shaders of the gl renderer live under
/org/gtk/libgsk/gl. This is purely cosmetic.

Merge branch 'matthiasc/for-master' into 'master'

node test: Improve --generate

See merge request GNOME/gtk!3397

settings: Make font-size changes apply immediately

We need to invalidate the style when font-size changes,
because we propagate this value through the initial
value of the CSS font-size property, and it will not
be recomputed otherwise.

immulticontext: Unset client widget on delegate change

Forgetting to do so was causing the Wayland im context
to leave behind a dead event controller. This was showing
up as a crash when closing the inspector after changing
the im-module property of a GtkText widget. The crash
was delayed until closing the inspector because the
inspector keeps a ref on the event controllers of the
currently shown widget.

imwayland: Name event controllers

This helps with debugging.

widget-factory: Name an event controller

This helps with debugging.

text: Name all event controllers

This helps with debugging.

inspector: Support copying nodes to clipboard

Add a button that copies the serialization of the
selected node to the clipboard.

node-editor: Support pasting the image

Add a button that copies the image to the clipboard.

nodeparser: Change the serialization

Arrange things so that non-child parameters
are always printed before the children. This
greatly helps with readability, which really
suffers when there's hundreds of lines of indented
children between the node start and its parameters.

Update all affected tests.

node test: Improve --generate

This is a pet peeve of mine: When we call
g_test_init() before handling --generate,
the random seed spew pollutes the output.
Highly annoying. I've fixes many test binaries
over the years, but more keep popping up.

Merge branch 'fix-downscaled-textures' into 'master'

ngl: Fix downscaled textures

See merge request GNOME/gtk!3395